Skip to content

chore: optionally add organization logo via lfx serve api (CM-1102)#4008

Merged
joanagmaia merged 4 commits intomainfrom
chore/lfx-serve-api-organization-logo
Apr 8, 2026
Merged

chore: optionally add organization logo via lfx serve api (CM-1102)#4008
joanagmaia merged 4 commits intomainfrom
chore/lfx-serve-api-organization-logo

Conversation

@joanagmaia
Copy link
Copy Markdown
Contributor

@joanagmaia joanagmaia commented Apr 7, 2026

This pull request introduces support for an optional organization logo in the organization creation API and updates the handling of organization attribute sources to include LFX_SERVE. The changes ensure that the logo can be provided when creating an organization and that the new attribute source is recognized throughout the system.

API changes:

  • Added an optional logo field to the createOrganization API request body schema and response, allowing clients to specify a logo when creating an organization. [1] [2]

Attribute source updates:

  • Added LFX_SERVE as a new value to the OrganizationAttributeSource enum, enabling it as a valid source for organization attributes.
  • Updated the ORG_DB_ATTRIBUTE_SOURCE_PRIORITY array to include LFX_SERVE, ensuring it is considered when determining attribute precedence.

Note

Medium Risk
Introduces a new organization attribute source and changes attribute source priority, which can alter which data wins during org attribute resolution; also expands the public create-org API surface area.

Overview
The public createOrganization endpoint now accepts an optional logo, passes it through to findOrCreateOrganization, and includes it in the creation response.

Organization creation via this endpoint is re-sourced from CUSTOM to the new OrganizationAttributeSource.LFX_SERVE, and the types + attribute-source priority list are updated to recognize LFX_SERVE ahead of enrichment sources when resolving organization attributes.

Reviewed by Cursor Bugbot for commit 5750de3. Bugbot is set up for automated code reviews on this repo. Configure here.

Signed-off-by: Joana Maia <jmaia@contractor.linuxfoundation.org>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds support for passing an optional organization logo during public API organization creation, and introduces LFX_SERVE as a recognized organization attribute source with defined precedence.

Changes:

  • Added LFX_SERVE to OrganizationAttributeSource.
  • Included LFX_SERVE in ORG_DB_ATTRIBUTE_SOURCE_PRIORITY.
  • Extended the public createOrganization endpoint to accept and forward an optional logo, and include it in the response.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
services/libs/types/src/enums/organizations.ts Adds LFX_SERVE to the organization attribute source enum.
services/libs/data-access-layer/src/organizations/attributesConfig.ts Adds LFX_SERVE into the attribute source priority list.
backend/src/api/public/v1/organizations/createOrganization.ts Accepts optional logo, uses LFX_SERVE as the org source, and returns logo in the response.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread backend/src/api/public/v1/organizations/createOrganization.ts Outdated
Comment thread backend/src/api/public/v1/organizations/createOrganization.ts
joanagmaia and others added 2 commits April 7, 2026 14:20
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Joana Maia <jmaia@contractor.linuxfoundation.org>
…rganization-logo

Signed-off-by: Joana Maia <jmaia@contractor.linuxfoundation.org>
@joanagmaia joanagmaia merged commit 45dd28a into main Apr 8, 2026
15 checks passed
@joanagmaia joanagmaia deleted the chore/lfx-serve-api-organization-logo branch April 8, 2026 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants